@chenyx113 , there are too many files to review.
please split to "single context of changes" per PR,
like, introduce README, introduce CMakeLists with main file.
I'll request split PRs until PR itself is simple.

@chenyx113 , there are too many files to review. please split to "single context of changes" per PR, like, introduce README, introduce CMakeLists with main file. I'll request split PRs until PR itself is simple.

@seanshpark Thanks for suggestion, because it is the first time uploading for the whole project, so it have many files. I understand it is difficult to review so many files at one PR.

but if split it to small changes, it will take many PRs for this submitting, is there any other suggestions for baseline code merge?

@seanshpark
according to your suggestion, I checked our current code, and have plan as follow, please help check.
step 1: project set up with Readme.md only, to introduce the project feature and user guide
step 2: submit the onnx-subgraph parser lib related code with CMakeLists, it will include more cpp files, I think we should make sure the building success at each submitting
step 3: add main.cpp & config.json and show how to use the onnx-subgraph-parser lib APIs, and can generate onnx-subgraph exe
step 4: add test models downloading and pre-processing scripts for user testing
step 5: Add python files for split original onnx model to subgraph models accroding to onnx-subgraph parsing result
step 6: Add verification python code to compare the inference result from original onnx model and spitted subgraph models, to make sure the model splitting is right

step 2 will have more files, and other steps will be simple, hope for your opinion.

Thanks & BR

according to your suggestion, I checked our current code, and have plan as follow, please help check.

I don't know the details of the plan. The basic rule is to put single context of changes and not much (not more than 50 lines) to review or your PR will have too many comments that will not be appropriate for you and the reviewers to communicate.

Q) is current PR has full changes? or is it a part ?
title has baseline and I assume there are more.
As I wrote on the e-mail, please put full changes so that when I see codes that I don't understand the reason, I can reference the full changes, not to ask dumb questions.

Q) is current PR has full changes? or is it a part ? title has baseline and I assume there are more. As I wrote on the e-mail, please put full changes so that when I see codes that I don't understand the reason, I can reference the full changes, not to ask dumb questions.

@seanshpark
it is the full changes, includes all of the code of onnx-subgraph tool, can be tested as the guideline in the Readme.md.
thank you:)